Repository navigation
Check Node.js at session start in the plugin - #18
Merged
Merged
Conversation
The commands create printed failed after npx coursecode create when coursecode wasn't installed globally, so they now use npx. New projects also point people to their AI tool, start at version 1.0.0, and no longer carry the template author's name. The README leads with the plugin. New projects use ESLint 10, create no longer prints npm's audit and funding notices, the narration help lists all five providers, and the npm homepage points to the site. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The plugin's MCP server needs Node.js to start, so when it was missing the CourseCode tools just failed to appear and the launcher's error never ran. A SessionStart hook now tells the AI tool to use the setup skill when Node.js is missing or too old. The hook needs no Node.js and stays silent when the version is fine. The hook runs in sh, Git Bash, and PowerShell, so CI now tests it on a Windows runner in Git Bash and both versions of PowerShell. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The plugin's MCP server needs Node.js to start. When Node.js was missing, the CourseCode tools just failed to appear, and the launcher's helpful error never ran because it needs Node.js too.
This adds a Claude Code
SessionStarthook that needs no Node.js. When Node.js is missing or older than 22.15, it tells the AI tool to use thesetupskill, which asks before installing anything. When Node.js is fine, it prints nothing.The same file,
plugin/hooks/check-node.cmd, works as a shell script and a Windows batch file. A new CI job runs it on a Windows runner.Merge #17 first. This branch is built on it, so its commit shows here until then.
Related Issue
None.
Type of Change
Checklist
npm run lintand it passesnpm run previewPreview doesn't apply because this only changes the plugin. The commit follows the repository's recent commit style.
Testing
createtests: 43 passednpm run lintandclaude plugin validate pluginpassedPATHand with a too-old versionscripts/smoke-plugin-session-check.ps1runs in CI with supported, old, and missing Node.js, in Git Bash, Windows PowerShell, and PowerShell 7Not tested: a real AI tool acting on the message. The Windows check runs the hook's command directly, not through Claude Code.
Screenshots (if applicable)
Not applicable.
🤖 Generated with Claude Code